Skip to content

Add maplist filter window and environment filtering#310

Open
AroPix wants to merge 3 commits into
lythx:devfrom
AroPix:feature/maplist-filtering
Open

Add maplist filter window and environment filtering#310
AroPix wants to merge 3 commits into
lythx:devfrom
AroPix:feature/maplist-filtering

Conversation

@AroPix

@AroPix AroPix commented Feb 19, 2026

Copy link
Copy Markdown
Contributor

Adds a window to filter maps:
Bildschirmfoto_20260219_201829

Also adds a argument to filter for Environment, if the server is not Nations packmask.
/list $e Snow as example.

I'm open to feedback. :)

@AroPix

AroPix commented Feb 19, 2026

Copy link
Copy Markdown
Contributor Author

I just noticed, it should also be able to handle the new custom envs, but ill add that at some point if this pr is even desired to get approved.

@wsrvn wsrvn left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm besides a few nitpicks, maybe some more descriptive method names could be beneficial too, but since they are used only within this file it should be fine

Comment thread plugins/maplist/ui/MaplistFilters.components.ts Outdated
Comment thread plugins/maplist/ui/MaplistFilters.components.ts
@AroPix AroPix force-pushed the feature/maplist-filtering branch from 974bd0f to 68a4c9c Compare March 3, 2026 19:05
@AroPix

AroPix commented Mar 3, 2026

Copy link
Copy Markdown
Contributor Author
image this is how it looks like with nations packmask btw (we only have Stadium maps then)

I also forgot to add the Filters to the navbar (like it is on my servers), so i added that aswell:
image

It also supports the custom envs now, but i removed NewSnowCar, as it is not a env but just a car anyway.

@wsrvn wsrvn left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm @lythx

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants